Skip to content

chore(deps): Update npm deps, add Node 20 to CI#86

Merged
dpogue merged 1 commit intoapache:masterfrom
dpogue:deps-202403
Apr 15, 2024
Merged

chore(deps): Update npm deps, add Node 20 to CI#86
dpogue merged 1 commit intoapache:masterfrom
dpogue:deps-202403

Conversation

@dpogue
Copy link
Copy Markdown
Member

@dpogue dpogue commented Mar 26, 2024

Platforms affected

All

Motivation and Context

Update some dependencies.

Description

  • Add node 20 to CI
  • Update GitHub Actions dependencies
  • Rebuild package-lock to resolve npm audit warnings
  • Replace fs-extra dependency with Node core fs API

Testing

Unit tests pass.

Checklist

  • I've run the tests to see all new and existing tests pass

erisu
erisu approved these changes Mar 26, 2024
View reviewed changes
Copy link
Copy Markdown
Member

@erisu erisu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dpogue dpogue marked this pull request as ready for review March 29, 2024 08:03
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 15, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 85.71429% with 1 line in your changes missing coverage. Please review.

Project coverage is 89.70%. Comparing base (3280edc) to head (d1c9c4c).
Report is 5 commits behind head on master.

Files with missing lines Patch % Lines
index.js 85.71% 1 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##           master      #86      +/-   ##
==========================================
+ Coverage   89.55%   89.70%   +0.15%     
==========================================
  Files           1        1              
  Lines          67       68       +1     
==========================================
+ Hits           60       61       +1     
  Misses          7        7

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

erisu
erisu approved these changes Apr 15, 2024
View reviewed changes
Copy link
Copy Markdown
Member

@erisu erisu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dpogue dpogue merged commit 254a49a into apache:master Apr 15, 2024
@dpogue dpogue deleted the deps-202403 branch April 15, 2024 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@erisu erisu erisu approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@dpogue @codecov-commenter @erisu